Skip to content

Conversation

@bmorelli25
Copy link
Member

@bmorelli25 bmorelli25 commented Feb 26, 2025

What

  • Adds codeowners
  • Updates issue template ordering, wording, and labeling
  • Updates project automation to support new project boards

How issues/triaging will work

  • Elastic employees will use the Internal Docs Template. This template requires more inputs to better scope issues for writers to wrk on.
  • Community members will use the Community Docs Template. This template requires less input as we want it to be easy for community members to contribute.
  • There are two additional templates for specific types of feedback:
    • API docs feedback template
    • UI text template
  • Finally, there is a website feedback template that is only for issues coming directly from our published docs

If a writer opens an issue in their teams area, they are expected to remove the needs-team label and apply the appropriate docs team labels. For all other instances of issues opened, the assigned "on call" writer will triage issues by adding them to their teams product boards and fixing easy issues.

@bmorelli25 bmorelli25 requested a review from a team as a code owner February 26, 2025 21:57
Copy link
Contributor

@georgewallace georgewallace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left my comments

@bmorelli25 bmorelli25 requested a review from a team as a code owner May 9, 2025 19:28
@bmorelli25 bmorelli25 changed the title Update Codeowners file Update Codeowners & Project automation May 9, 2025
Copy link
Contributor

@georgewallace georgewallace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would like @shainaraskas and @szabosteve to review the changes to the groups. I think I got it right.

bmorelli25 and others added 3 commits May 19, 2025 13:25
@bmorelli25 bmorelli25 changed the title Update Codeowners & Project automation Update Codeowners, Issues, and Project automation May 19, 2025
Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Experience docs is missing from a few places

Co-authored-by: florent-leborgne <[email protected]>
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

couple of mild quibbles

Comment on lines +60 to +62
# Legacy serverless docs
/serverless/ @elastic/admin-docs
/serverless/serverless-changelog.asciidoc @elastic/docs
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can just delete these files, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk maybe, probably?

Copy link
Contributor

@georgewallace georgewallace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good with this once we add @shainaraskas feedback

Copy link
Contributor

@colleenmcginnis colleenmcginnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple little suggestions.

@bmorelli25 bmorelli25 enabled auto-merge (squash) May 20, 2025 21:12
@bmorelli25
Copy link
Member Author

Thank you for all of your helpful suggestions 👥

@bmorelli25 bmorelli25 disabled auto-merge May 20, 2025 21:17
@bmorelli25 bmorelli25 merged commit 4701b33 into main May 20, 2025
4 of 5 checks passed
@bmorelli25 bmorelli25 deleted the add-codeowners branch May 20, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants